Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add automatic versioning and required files #168

Merged
merged 3 commits into from
Feb 4, 2021

Conversation

lsetiawan
Copy link
Contributor

Overview

This PR add automatic versioning. This also adds requirements.txt as well as LICENSE file to source package since it's missing and causes conda-forge/staged-recipes#13866 to fail.

@lsetiawan
Copy link
Contributor Author

@JessicaS11 Please review and merge 😄

setup.py Outdated Show resolved Hide resolved
Copy link
Member

@weiji14 weiji14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully @JessicaS11 can have a look before merging. This PR looks pretty much ok to me aside from two minor comments. We should try and tag a v0.3.3 release soon after and get that icepyx conda package sorted at conda-forge/staged-recipes#13866!

.gitignore Outdated Show resolved Hide resolved
requirements.txt Show resolved Hide resolved
@codecov-io
Copy link

Codecov Report

Merging #168 (827f5a7) into development (9446737) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #168      +/-   ##
===============================================
+ Coverage        47.87%   47.92%   +0.04%     
===============================================
  Files               17       17              
  Lines             1178     1179       +1     
  Branches           262      262              
===============================================
+ Hits               564      565       +1     
  Misses             574      574              
  Partials            40       40              
Impacted Files Coverage Δ
icepyx/__init__.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9446737...827f5a7. Read the comment docs.

@JessicaS11 JessicaS11 merged commit ac78f81 into icesat2py:development Feb 4, 2021
@lsetiawan lsetiawan deleted the add_versioneer branch February 4, 2021 01:34
@lsetiawan lsetiawan mentioned this pull request Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants